Skip to content

Keep cufile.log out of the Java source tree - #2553

Merged
rapids-bot[bot] merged 2 commits into
NVIDIA:release/26.10from
imotov:move-cufile-during-java-tests
Sep 9, 2026
Merged

Keep cufile.log out of the Java source tree#2553
rapids-bot[bot] merged 2 commits into
NVIDIA:release/26.10from
imotov:move-cufile-during-java-tests

Conversation

@imotov

@imotov imotov commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

cuFile writes cufile.log into the JVM's working directory, leaving a stray file in java/cuvs-java and java/cuvs-lucene after every test run. Point CUFILE_LOGFILE_PATH at target/ in the failsafe and surefire forks, and gitignore the file for the example and benchmark modules, which are launched as bare JVMs and so aren't covered by the plugin config.

Follow-up to #2257

@imotov
imotov requested a review from a team as a code owner September 4, 2026 18:42
@imotov imotov added bug Something isn't working non-breaking Introduces a non-breaking change Java Lucene labels Sep 4, 2026
@imotov imotov self-assigned this Sep 4, 2026
@imotov
imotov changed the base branch from main to release/26.10 September 8, 2026 23:08
cuFile writes cufile.log into the JVM's working directory, leaving a stray file in java/cuvs-java and java/cuvs-lucene after every test run. Point CUFILE_LOGFILE_PATH at target/ in the failsafe and surefire forks, and gitignore the file for the example and benchmark modules, which are launched as bare JVMs and so aren't covered by the plugin config.

Follow-up to NVIDIA#2257
@imotov
imotov force-pushed the move-cufile-during-java-tests branch from 6443c36 to c12bc39 Compare September 8, 2026 23:14
@imotov

imotov commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit c01c81a into NVIDIA:release/26.10 Sep 9, 2026
67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Java Lucene non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants